Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

EPUB/Snapshot: Override print options to correct title & URL #149

Merged

Conversation

AbeJellinek
Copy link
Member

EPUBs can show the book title (disabled by default) but don't have any URL. Snapshots show the page title and the URL they were imported from.

Needs zotero/zotero#4820, fixes zotero/zotero#4818

@AbeJellinek AbeJellinek requested a review from mrtcode November 6, 2024 20:08
@mrtcode
Copy link
Member

mrtcode commented Nov 7, 2024

This specific PR looks good.

However, I have an issue with one EPUB file where all printed pages are empty (I will send a link).

@AbeJellinek AbeJellinek force-pushed the 4818-epub-snapshot-print-title-url branch from 7f30a1d to 5c9dfda Compare November 7, 2024 20:47
@AbeJellinek AbeJellinek force-pushed the 4818-epub-snapshot-print-title-url branch from 5c9dfda to a47febd Compare November 7, 2024 20:50
@AbeJellinek AbeJellinek merged commit c64aaa3 into zotero:master Nov 8, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Printed EPUBs/snapshots include "Zotero" and "about:srcdoc" in page header
2 participants